ci: retry pkg-pr-new publish on transient 5xx#1500
ci: retry pkg-pr-new publish on transient 5xx#1500TheodorNEngoy wants to merge 2 commits intomodelcontextprotocol:mainfrom
Conversation
|
@modelcontextprotocol/client
@modelcontextprotocol/server
@modelcontextprotocol/express
@modelcontextprotocol/hono
@modelcontextprotocol/node
commit: |
|
Context: this was prompted by transient pkg.pr.new 5xx/Cloudflare errors intermittently failing the required "Publish Any Commit" check (e.g. on #1499). With this retry in place, those flakes should stop blocking otherwise-green PRs. |
|
Note: no changeset here since this is CI/workflow-only (no package output changes expected). |
|
@claude review |
felixweinberger
left a comment
felixweinberger
left a comment
There was a problem hiding this comment.
Thanks for this. The regex is over-escaped: change \\(5[0-9]{2}\\) to \(5[0-9]{2}\) (single backslash). With -E, double backslash matches a literal backslash, so the current pattern looks for Check failed \\(502\\) which pkg-pr-new never outputs.
2 participants
The
Publish Any Commitworkflow usespkg-pr-newto publish preview packages for each PR.We’ve seen this step intermittently fail with
pkg.pr.new5xx / Cloudflare worker errors (transient external failures), which blocks otherwise-green PRs.This change wraps the publish step with a small retry loop:
pkg.pr.new/Cloudflare 5xx failure.No behavior changes to the SDK itself.